Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

azurerm_security_center_setting - Support for Sentinel security setting in resource azurerm_security_center_setting #16882

Closed
wants to merge 2 commits into from

Conversation

sinbai
Copy link
Contributor

@sinbai sinbai commented May 20, 2022

Fix issue #16846.

Test Results:

PASS: TestAccSecurityCenterSetting_basic (226.87s)
PASS: TestAccSecurityCenterSetting_requiresImport (131.64s)
PASS: TestAccSecurityCenterSetting_update (622.76s)

Comment on lines +54 to +63
"kind": {
Type: pluginsdk.TypeString,
Optional: true,
ForceNew: true,
Default: security.KindDataExportSettings,
ValidateFunc: validation.StringInSlice([]string{
string(security.KindDataExportSettings),
string(security.KindAlertSyncSettings),
}, false),
},
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

rather than exposing a kind field this wants to be split into two different resources - as the discriminator implementations have different validation requirements etc

can you take a look into splitting these out @sinbai?

@sinbai sinbai changed the title azurerm_security_center_setting : Support for Sentinel security setting in resource azurerm_security_center_setting azurerm_security_center_setting - Support for Sentinel security setting in resource azurerm_security_center_setting Aug 15, 2022
@katbyte
Copy link
Collaborator

katbyte commented Sep 24, 2022

@sinbai - i'm going to close this as its been 3 months without any activity - please do re open this when its ready for review again!

@katbyte katbyte closed this Sep 24, 2022
@github-actions
Copy link

I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active contributions.
If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Oct 25, 2022
@sinbai sinbai deleted the securitycenter/fix_issue_16846 branch March 28, 2024 03:05
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants